-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[#20] Task 등록, 수정, 삭제 기능에 Reminder 설정 기능 추가 #21
Conversation
.build(); | ||
} | ||
|
||
public void updateLastReminderSentTime(LocalDateTime lastReminderSentTime, ClockHolder clockHolder) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저 예전부터 궁금하던게 있는데요. clockHolder는 어떤 역할을 하는 것인가요?
|
||
@RequiredArgsConstructor | ||
@Service | ||
public class ReminderNotificationService { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 로직은 좀 더 고민해보고, 어떻게 할지 생각해보죠. :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넵 지난시간에 말씀해주신 방법과 함께 고민해보겠습니다 ㅎㅎ
Assertions.assertTrue(reminderSettings.isReminderDue(new TestClockHolder(currentDateTime))); | ||
}, | ||
() -> { | ||
LocalDateTime currentDateTime = taskStartDateTime.plusMinutes(3); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아항 ~ 요런 느낌이었군요 :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고 하셨습니다.
리뷰감사합니당 ㅎㅎ |
배경
알림 요구사항
변경 내용
참고내용
이전 리뷰내용과 ClockHolder 내용은 아직 반영 전입니다..! 이번주중에 나누어서 반영하도록 하겠습니다. 🙂